Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: modified unbalance reactions #816

Conversation

JHL-452b
Copy link
Collaborator

@JHL-452b JHL-452b commented Apr 25, 2024

Main improvements in this PR:

As proposed in #814 and #813:

  • change the charge of mets MAM20004n, MAM20006c, MAM20018r, MAM20021i, MAM20024r, MAM20025i, MAM20026r, MAM20027i, MAM20028r, MAM20029i, MAM20030r, MAM20031i, MAM20041r, MAM20042c, MAM20043r, MAM20044c, MAM20056r, MAM20057c from 0 to -1

  • remove MAM02039e in MAR01444;

  • remove MAM02039r in MAR20021;

  • remove MAM02039i in MAR20022;

  • remove MAM02039r in MAR20023;

  • remove MAM02039i in MAR20024;

  • remove MAM02039r in MAR20025;

  • remove MAM02039i in MAR20026;

  • remove MAM02039r in MAR20027;

  • remove MAM02039i in MAR20028;

  • remove MAM02039r in MAR20029;

  • remove MAM02039i in MAR20030;

  • add MAM02039m in MAR20111 as a product;

  • add MAM01422m in MAR20112 as a product;

  • add MAM02039c in MAR03481 as a product;

  • replace the formula of MAM01657r to "C40H65O7P2"

  • replace the formula of MAM20082m from None to "C11H23N1O2S2"

  • replace MAM02041x with MAM01803x in MAR08415

  • replace MAM02630x with MAM01802x in MAR08415

  • replace MAM02041x with MAM01803x in MAR03984

  • replace MAM02630x with MAM01802x in MAR03984

  • replace MAM02041x with MAM01803x in MAR12372

  • replace MAM02630x with MAM01802x in MAR12372

I hereby confirm that I have:

  • Tested my code on my own computer for running the model
  • Selected develop as a target branch
  • Any removed reactions and metabolites have been moved to the corresponding deprecated identifier lists

Copy link
Collaborator

@feiranl feiranl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@feiranl feiranl merged commit 56b345d into SysBioChalmers:develop Apr 26, 2024
4 checks passed
@JHL-452b JHL-452b mentioned this pull request May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants